Skip to content

[TRNF-3299] add return action to sdk#88

Merged
npcastro merged 1 commit into
masterfrom
TRNF-3299-transfer-returns
Jun 25, 2025
Merged

[TRNF-3299] add return action to sdk#88
npcastro merged 1 commit into
masterfrom
TRNF-3299-transfer-returns

Conversation

@npcastro

Copy link
Copy Markdown
Contributor

Summary

This PR adds support for the return transfer endpoint to the Fintoc Python SDK.

Usage Example

from fintoc import Fintoc

fintoc = Fintoc("your-api-key")

# Create a checkout session
transfer = fintoc.transfers.return(
    transfer_id='tr_...',
)

@npcastro npcastro force-pushed the TRNF-3299-transfer-returns branch from bc6318a to 3d0e35b Compare June 24, 2025 22:45
Comment thread fintoc/managers/v2/transfers_manager.py Outdated
methods = ["list", "get", "create"]
methods = ["list", "get", "create", "return"]

def _return(self, **kwargs):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solo return, sin el _

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cacha que no se puede. Choca con return de python.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto me lo sugirió claude. Voy a ver si hay otra salida.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lo dejaría al revés entonces! .return_. Es que si va antes es como indicar que el método es privado? qué opinas @daleal ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

100% return_

@npcastro npcastro force-pushed the TRNF-3299-transfer-returns branch from 3d0e35b to 3f5be44 Compare June 24, 2025 23:17
@npcastro npcastro merged commit a401636 into master Jun 25, 2025
5 checks passed
@npcastro npcastro deleted the TRNF-3299-transfer-returns branch June 25, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants